Skip to content

mujoco3.12 - #25

Closed
k1000dai wants to merge 2 commits into
enactic:mainfrom
k1000dai:dev/3.12
Closed

mujoco3.12#25
k1000dai wants to merge 2 commits into
enactic:mainfrom
k1000dai:dev/3.12

Conversation

@k1000dai

@k1000dai k1000dai commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

On MuJoCo 3.12 every IK solve fails with ArmJointLimit only supports scalar joints.
(20 of 41 tests).

model.jnt_type[j] is an np.int32, and x in (a, b) compares
a == x
— the enum on the left. mjJNT_HINGE == np.int32(3) was True up to 3.10 and is
False on 3.12, so the scalar-joint check rejects every arm joint.

Cause: MuJoCo 3.12 bumped its pinned pybind11
v3.0.4 → v3.1.0,
which rewrote py::enum_ comparisons (#5887);
NumPy scalars now fall through __eq__ to a const object & overload returning false.
MuJoCo's own enum bindings are unchanged, and this is not in the 3.12 release notes.

Fixed by comparing int to int. The added test fails on both 3.10 and 3.12 without it.
42 tests pass on MuJoCo 3.6.0 (floor), 3.10.0, and 3.12.0; no floor raised.

Copilot AI lite review requested due to automatic review settings August 24, 2026 08:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread uv.lock Outdated
Copilot AI review requested due to automatic review settings August 24, 2026 08:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kou

kou commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

I close this in favor of #26.

@kou kou closed this Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants